Skip to content

fix(a11y): label account QR code button#60616

Open
Will-thom wants to merge 3 commits into
nextcloud:masterfrom
Will-thom:issue-60292-a11y-header-buttons
Open

fix(a11y): label account QR code button#60616
Will-thom wants to merge 3 commits into
nextcloud:masterfrom
Will-thom:issue-60292-a11y-header-buttons

Conversation

@Will-thom

Copy link
Copy Markdown

Summary

  • Add an accessible label to the account menu QR code icon-only button.
  • Add coverage for the QR code button accessible name.
  • Add coverage that the app menu trigger buttons expose accessible labels.

Fixes #60292

Tests

  • npx vitest run --project nextcloud-ui-legacy core/src/tests/components/AccountMenuProfileEntry.spec.ts core/src/tests/components/AppMenu.spec.ts
  • npm run lint
  • git diff --check origin/master...HEAD

Validated in a clean Docker container from a fresh nextcloud/server clone using node:24-bookworm.

@Will-thom Will-thom requested a review from a team as a code owner May 20, 2026 15:39
@Will-thom Will-thom requested review from nfebe, sorbaugh and susnux and removed request for a team May 20, 2026 15:39

@susnux susnux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code looks good, thank you

@susnux susnux requested review from pringelmann and szaimen and removed request for sorbaugh May 20, 2026 15:48
@susnux susnux added bug 3. to review Waiting for reviews labels May 20, 2026
@susnux susnux added this to the Nextcloud 35 milestone May 20, 2026

@szaimen szaimen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐘

@CarlSchwan

Copy link
Copy Markdown
Member

/compile amend /

@CarlSchwan

Copy link
Copy Markdown
Member

Your commit is missing the DCO

@Will-thom Will-thom force-pushed the issue-60292-a11y-header-buttons branch from 7c3cdc0 to 3e65513 Compare May 23, 2026 03:47
@AndyScherzinger AndyScherzinger force-pushed the issue-60292-a11y-header-buttons branch from 3e65513 to 587fc62 Compare June 1, 2026 21:36
@AndyScherzinger

Copy link
Copy Markdown
Member

/compile

@nextcloud nextcloud deleted a comment from backportbot Bot Jun 1, 2026
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

@AndyScherzinger AndyScherzinger force-pushed the issue-60292-a11y-header-buttons branch from 587fc62 to 746dc6c Compare June 4, 2026 05:31
@AndyScherzinger

Copy link
Copy Markdown
Member

/compile

Will-thom added 2 commits June 6, 2026 02:28
Signed-off-by: Francisco Silva <116388885+Will-thom@users.noreply.github.com>
Signed-off-by: Will-thom <116388885+Will-thom@users.noreply.github.com>
@Will-thom Will-thom force-pushed the issue-60292-a11y-header-buttons branch from 3a6368e to 685ce73 Compare June 6, 2026 05:46
Signed-off-by: Will-thom <116388885+Will-thom@users.noreply.github.com>
@susnux susnux added the community pull requests from community label Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug community pull requests from community feedback-requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NcButton accessibility warning fires twice on every logged-in page load (UserMenu, MainMenu)

5 participants